Skip to content

Conversation

@lukas-krecan
Copy link
Contributor

Added example how to use JsonUnit.

Added example how to use JsonUnit.
@lukas-krecan
Copy link
Contributor Author

I am not sure if I am supposed to add the example to Spring test or what's expected. Feel free to send suggenstions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Nov 24, 2024
Copy link
Member

@snicoll snicoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a couple of comments.

If you prefer to use another library, you can provide an implementation of
{spring-framework-api}/test/json/JsonComparator.html[`JsonComparator`].

Alternatively, you can use AssertJ https://github.com/assertj/assertj/blob/3.x/assertj-core/src/main/java/org/assertj/core/api/AssertFactory.java[AssertFactory].
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rephrase that to be consistent with previous sentences. For instance.

JsonUnit is also supported via the standard convertTomethod. I am not keen to start referring to AssertJ's Javadoc for this.

do this

```
import static net.javacrumbs.jsonunit.assertj.JsonAssertions.jsonUnitJson;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't markdown. We don't compile against JSonUnit so we can't add the example.

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Nov 26, 2024
@snicoll snicoll self-assigned this Nov 26, 2024
@snicoll
Copy link
Member

snicoll commented Dec 6, 2024

Closing due to the lack of feedback.

@snicoll snicoll closed this Dec 6, 2024
@snicoll snicoll removed status: waiting-for-feedback We need additional information before we can continue status: waiting-for-triage An issue we've not yet triaged or decided on labels Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants